Skip to content
This repository was archived by the owner on Dec 19, 2023. It is now read-only.

Feature/726 fix graphiql config #777

Merged
merged 3 commits into from
Jan 24, 2022
Merged

Conversation

oliemansm
Copy link
Member

Based on BlasiusSecundus branch

BlasiusSecundus and others added 2 commits October 13, 2021 21:18
There was some inconsistency in configuration handling. The
`GraphiQLProperties` were not used for headers and GraphiQL props.
Instead, the properties were manually loaded.

This part, however, was not updated with the starter reorganization,
and expected the variables in the old place
(`graphiql.props.variables.`).

This commit fixes this by using the props / headers from the
configuration properties.
@github-actions
Copy link

github-actions bot commented Jan 22, 2022

Unit Test Results

  91 files    91 suites   44s ⏱️
248 tests 248 ✔️ 0 💤 0
253 runs  253 ✔️ 0 💤 0

Results for commit a5988b1.

♻️ This comment has been updated with latest results.

@oliemansm oliemansm merged commit c4faf03 into master Jan 24, 2022
@oliemansm oliemansm added this to the 12.0.1 milestone Jan 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants